fix(orchestrator-form-widgets): show spinner immediately on ActiveText retrigger by lokanandaprabhu · Pull Request #2279 · redhat-developer/rhdh-plugins

lokanandaprabhu · 2026-02-09T09:35:45Z

Hey, I just made a Pull Request!

Fixes:

https://issues.redhat.com/browse/RHDHBUGS-2594

PR description:

Problem

ActiveText widgets with fetch:retrigger (and no fetch:url) briefly show stale ui:text during the debounce window. The spinner only appears once evaluation starts, which is visible in production latency.

Solution

Start loading immediately when retrigger dependencies change (before debounce).
Keep spinner visible through fetch/evaluation completion.
Clear widget values on retrigger when fetch:clearOnRetrigger is enabled to avoid stale state during refetch.

Changes

useFetch: mark loading as soon as retrigger deps change and are satisfied.
useFetchAndEvaluate: mark loading immediately on retrigger change even without fetch.
Add fetch:clearOnRetrigger support for ActiveTextInput, ActiveDropdown, and ActiveMultiSelect.

✔️ Checklist

A changeset describing the change and affected packages. (more info)
Added or Updated documentation
Tests for new functionality and regression tests for bug fixes
Screenshots attached (for UI changes)

rhdh-gh-app · 2026-02-09T09:36:08Z

Changed Packages

Package Name	Package Path	Changeset Bump	Current Version
@red-hat-developer-hub/backstage-plugin-orchestrator-form-widgets	workspaces/orchestrator/plugins/orchestrator-form-widgets	patch	`v1.7.1`

lokanandaprabhu · 2026-02-09T12:34:46Z

/hold

lokanandaprabhu · 2026-02-12T06:14:35Z

Test Spinner ActiveText:

Minimal workflow to validate ActiveText shows a spinner immediately when retrigger inputs change.
Uses a delayed fetch to make the debounce window visible and confirm the spinner stays until evaluation completes.

test-spinner-active-text.sw.yaml

id: test-spinner-active-text
version: "1.0"
specVersion: "0.8"
name: Test Spinner ActiveText
description: Simple workflow to test ActiveText spinner on retrigger
dataInputSchema: schemas/test-spinner-active-text__main-schema.json
start: LogState
functions:
  - name: logFunction
    type: custom
    operation: sysout
states:
  - name: LogState
    type: operation
    actions:
      - name: logMessage
        functionRef:
          refName: logFunction
          arguments:
            message: "Workflow started with input: ${.}"
    end: true

schemas/test-spinner-active-text__main-schema.json

{
  "$id": "classpath:/schemas/test-spinner-active-text__main-schema.json",
  "$schema": "http://json-schema.org/draft-07/schema#",
  "title": "Test Spinner ActiveText",
  "type": "object",
  "properties": {
    "step1": {
      "type": "object",
      "title": "Step 1 - ActiveText Spinner",
      "properties": {
        "inputNamespace": {
          "title": "Namespace",
          "type": "string",
          "default": "",
          "ui:widget": "ActiveTextInput"
        },
        "inputCluster": {
          "title": "Cluster",
          "type": "string",
          "default": "",
          "ui:widget": "ActiveTextInput"
        },
        "exists": {
          "type": "string",
          "default": "false",
          "ui:hidden": true,
          "ui:props": {
            "fetch:method": "GET",
            "fetch:url": "http://localhost:18080/delay?ms=5000",
            "fetch:response:value": "$string(true)",
            "fetch:response:default": "false",
            "fetch:error:ignoreUnready": true,
            "fetch:retrigger": [
              "current.step1.inputNamespace",
              "current.step1.inputCluster"
            ]
          },
          "ui:widget": "ActiveTextInput"
        },
        "existsMessage": {
          "type": "string",
          "ui:widget": "ActiveText",
          "ui:props": {
            "ui:text": "## The namespace \"$${{current.step1.inputNamespace}}\" does not exist on cluster \"$${{current.step1.inputCluster}}\"",
            "fetch:method": "GET",
            "fetch:url": "http://localhost:18080/delay?ms=5000",
            "fetch:retrigger": [
              "current.step1.inputNamespace",
              "current.step1.inputCluster"
            ]
          }
        }
      },
      "required": ["inputNamespace", "inputCluster"]
    }
  }
}

lokanandaprabhu · 2026-02-12T06:16:10Z

Test Spinner ActiveDropdown

Static dropdown to trigger ActiveText fetch on selection change without dropdown fetching.
Confirms the ActiveText spinner appears instantly on dropdown change and persists until the delayed fetch resolves.

test-spinner-active-dropdown.sw.yaml

id: test-spinner-active-dropdown
version: "1.0"
specVersion: "0.8"
name: Test Spinner ActiveDropdown
description: Simple workflow to test ActiveDropdown spinner on retrigger
dataInputSchema: schemas/test-spinner-active-dropdown__main-schema.json
start: LogState
functions:
  - name: logFunction
    type: custom
    operation: sysout
states:
  - name: LogState
    type: operation
    actions:
      - name: logMessage
        functionRef:
          refName: logFunction
          arguments:
            message: "Workflow started with input: ${.}"
    end: true

schemas/test-spinner-active-dropdown__main-schema.json

{
  "$id": "classpath:/schemas/test-spinner-active-dropdown__main-schema.json",
  "$schema": "http://json-schema.org/draft-07/schema#",
  "title": "Test Spinner ActiveDropdown",
  "type": "object",
  "properties": {
    "step1": {
      "type": "object",
      "title": "Step 1 - ActiveDropdown Spinner",
      "properties": {
        "dropdown": {
          "type": "string",
          "title": "Dropdown (Static)",
          "description": "Change this to retrigger the ActiveText fetch.",
          "enum": ["Option A", "Option B", "Option C"],
          "default": "Option A"
        },
        "dropdownMessage": {
          "type": "string",
          "title": "Dropdown Message",
          "ui:widget": "ActiveText",
          "ui:props": {
            "ui:text": "Selected option: \"$${{current.step1.dropdown}}\"",
            "fetch:method": "GET",
            "fetch:url": "http://localhost:18080/delay?ms=5000",
            "fetch:retrigger": ["current.step1.dropdown"]
          }
        }
      }
    }
  }
}

lokanandaprabhu · 2026-02-12T06:16:49Z

…mpletes

Co-authored-by: Cursor <cursoragent@cursor.com>

Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com>

Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com>

Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. Co-authored-by: Cursor <cursoragent@cursor.com>

sonarqubecloud · 2026-02-25T10:48:22Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

lokanandaprabhu · 2026-02-25T11:47:01Z

------BEFORE-------

Screen.Recording.2026-02-25.at.4.09.52.PM.mov

--------AFTER------

Screen.Recording.2026-02-25.at.3.47.11.PM.mov

lokanandaprabhu · 2026-02-25T11:47:17Z

/unhold

mareklibra · 2026-02-26T06:44:27Z

Thanks, it was tricky issue. It's good that this behavior is left to be optional, there are cases where the old flow makes sense.

lokanandaprabhu · 2026-02-26T06:53:55Z

/cherrypick orchestrator-1.8

openshift-cherrypick-robot · 2026-02-26T06:55:08Z

@lokanandaprabhu: #2279 failed to apply on top of branch "orchestrator-1.8":

Applying: fix: show spinner on ActiveText retrigger
Using index info to reconstruct a base tree...
M	workspaces/orchestrator/plugins/orchestrator-form-widgets/src/utils/useFetch.ts
M	workspaces/orchestrator/plugins/orchestrator-form-widgets/src/utils/useFetchAndEvaluate.ts
Falling back to patching base and 3-way merge...
Auto-merging workspaces/orchestrator/plugins/orchestrator-form-widgets/src/utils/useFetchAndEvaluate.ts
Auto-merging workspaces/orchestrator/plugins/orchestrator-form-widgets/src/utils/useFetch.ts
CONFLICT (content): Merge conflict in workspaces/orchestrator/plugins/orchestrator-form-widgets/src/utils/useFetch.ts
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 fix: show spinner on ActiveText retrigger

Details

In response to this:

/cherrypick orchestrator-1.8

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

…t retrigger (#2279) * fix: show spinner on ActiveText retrigger * fix(orchestrator-form-widgets): keep spinner until ActiveText eval completes * Merge upstream/main Co-authored-by: Cursor <cursoragent@cursor.com> * chore(changeset): mention clearOnRetrigger Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com> * refactor(orchestrator-form-widgets): dedupe clearOnRetrigger Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator-form-widgets): guard retrigger races Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com>

…t retrigger (#2279) (#2513) * fix: show spinner on ActiveText retrigger * fix(orchestrator-form-widgets): keep spinner until ActiveText eval completes * Merge upstream/main * chore(changeset): mention clearOnRetrigger Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. * refactor(orchestrator-form-widgets): dedupe clearOnRetrigger Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. * fix(orchestrator-form-widgets): guard retrigger races Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. --------- Co-authored-by: Lokananda Prabhu <102503482+lokanandaprabhu@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>

…t retrigger (#2279) (#2517) * fix: show spinner on ActiveText retrigger * fix(orchestrator-form-widgets): keep spinner until ActiveText eval completes * Merge upstream/main * chore(changeset): mention clearOnRetrigger Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. * refactor(orchestrator-form-widgets): dedupe clearOnRetrigger Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. * fix(orchestrator-form-widgets): guard retrigger races Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. --------- Co-authored-by: Lokananda Prabhu <102503482+lokanandaprabhu@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>

…t retrigger (#2279) * fix: show spinner on ActiveText retrigger * fix(orchestrator-form-widgets): keep spinner until ActiveText eval completes * Merge upstream/main Co-authored-by: Cursor <cursoragent@cursor.com> * chore(changeset): mention clearOnRetrigger Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com> * refactor(orchestrator-form-widgets): dedupe clearOnRetrigger Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator-form-widgets): guard retrigger races Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com>

* feat(orchestrator): add card height mode config for workflow run page (#2386) * feat(orchestrator): add card height mode config Expose a workflow instance page option to switch between fixed card heights and content-based sizing, with a new hook and changeset entry. Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator): simplify layout and warn on invalid mode Refactor the workflow instance layout to reuse card components and rename the height mode flag for clarity, and warn when config values are unexpected before falling back to fixed mode. Made-with: Cursor --------- Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator-form-widgets): show spinner immediately on ActiveText retrigger (#2279) * fix: show spinner on ActiveText retrigger * fix(orchestrator-form-widgets): keep spinner until ActiveText eval completes * Merge upstream/main Co-authored-by: Cursor <cursoragent@cursor.com> * chore(changeset): mention clearOnRetrigger Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com> * refactor(orchestrator-form-widgets): dedupe clearOnRetrigger Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator-form-widgets): guard retrigger races Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Lokananda Prabhu <102503482+lokanandaprabhu@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>

…t retrigger (redhat-developer#2279) * fix: show spinner on ActiveText retrigger * fix(orchestrator-form-widgets): keep spinner until ActiveText eval completes * Merge upstream/main Co-authored-by: Cursor <cursoragent@cursor.com> * chore(changeset): mention clearOnRetrigger Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com> * refactor(orchestrator-form-widgets): dedupe clearOnRetrigger Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator-form-widgets): guard retrigger races Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(orchestrator-form-widgets): show spinner immediately on ActiveText retrigger (#2279) * fix: show spinner on ActiveText retrigger * fix(orchestrator-form-widgets): keep spinner until ActiveText eval completes * Merge upstream/main Co-authored-by: Cursor <cursoragent@cursor.com> * chore(changeset): mention clearOnRetrigger Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com> * refactor(orchestrator-form-widgets): dedupe clearOnRetrigger Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator-form-widgets): guard retrigger races Ignore stale fetch responses when retrigger values change and avoid reapplying cached data while a retriggered fetch is loading. Use layout effect for clearOnRetrigger to reduce UI flicker. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Cursor <cursoragent@cursor.com> * feat(orchestrator): add card height mode config for workflow run page (#2386) * feat(orchestrator): add card height mode config Expose a workflow instance page option to switch between fixed card heights and content-based sizing, with a new hook and changeset entry. Co-authored-by: Cursor <cursoragent@cursor.com> * fix(orchestrator): simplify layout and warn on invalid mode Refactor the workflow instance layout to reuse card components and rename the height mode flag for clarity, and warn when config values are unexpected before falling back to fixed mode. Made-with: Cursor --------- Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: Lokananda Prabhu <102503482+lokanandaprabhu@users.noreply.github.com> Co-authored-by: Cursor <cursoragent@cursor.com>

lokanandaprabhu requested review from batzionb, gciavarrini and mareklibra as code owners February 9, 2026 09:35

openshift-ci bot added the do-not-merge/hold label Feb 9, 2026

lokanandaprabhu and others added 6 commits February 25, 2026 16:13

fix: show spinner on ActiveText retrigger

1d9550b

fix(orchestrator-form-widgets): keep spinner until ActiveText eval co…

30c7c1e

…mpletes

Merge upstream/main

807414f

Co-authored-by: Cursor <cursoragent@cursor.com>

chore(changeset): mention clearOnRetrigger

077296f

Document the new fetch:clearOnRetrigger behavior in the existing changeset for the ActiveText retrigger spinner update. Co-authored-by: Cursor <cursoragent@cursor.com>

refactor(orchestrator-form-widgets): dedupe clearOnRetrigger

442666e

Extract shared clear-on-retrigger behavior into a reusable hook and reuse it across ActiveTextInput, ActiveDropdown, and ActiveMultiSelect. Co-authored-by: Cursor <cursoragent@cursor.com>

lokanandaprabhu force-pushed the fix/active-text-retrigger-spinner branch from 53c8e9d to 2d96cc0 Compare February 25, 2026 10:47

openshift-ci bot removed the do-not-merge/hold label Feb 25, 2026

mareklibra approved these changes Feb 26, 2026

View reviewed changes

mareklibra merged commit d6dff3b into redhat-developer:main Feb 26, 2026
9 checks passed

lokanandaprabhu mentioned this pull request Feb 26, 2026

[orchestrator-1.8] fix(orchestrator-form-widgets): show spinner immediately on ActiveText retrigger #2402

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(orchestrator-form-widgets): show spinner immediately on ActiveText retrigger#2279

fix(orchestrator-form-widgets): show spinner immediately on ActiveText retrigger#2279
mareklibra merged 6 commits intoredhat-developer:mainfrom
lokanandaprabhu:fix/active-text-retrigger-spinner

lokanandaprabhu commented Feb 9, 2026 •

edited

Loading

Uh oh!

rhdh-gh-app bot commented Feb 9, 2026 •

edited

Loading

Uh oh!

lokanandaprabhu commented Feb 9, 2026

Uh oh!

lokanandaprabhu commented Feb 12, 2026

Uh oh!

lokanandaprabhu commented Feb 12, 2026

Uh oh!

lokanandaprabhu commented Feb 12, 2026

Uh oh!

sonarqubecloud bot commented Feb 25, 2026

Uh oh!

lokanandaprabhu commented Feb 25, 2026

Uh oh!

lokanandaprabhu commented Feb 25, 2026

Uh oh!

mareklibra commented Feb 26, 2026

Uh oh!

Uh oh!

lokanandaprabhu commented Feb 26, 2026

Uh oh!

openshift-cherrypick-robot commented Feb 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

lokanandaprabhu commented Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Hey, I just made a Pull Request!

Fixes:

PR description:

Problem

Solution

Changes

✔️ Checklist

Uh oh!

rhdh-gh-app bot commented Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changed Packages

Uh oh!

lokanandaprabhu commented Feb 9, 2026

Uh oh!

lokanandaprabhu commented Feb 12, 2026

Uh oh!

lokanandaprabhu commented Feb 12, 2026

Uh oh!

lokanandaprabhu commented Feb 12, 2026

Uh oh!

sonarqubecloud bot commented Feb 25, 2026

Quality Gate passed

Uh oh!

lokanandaprabhu commented Feb 25, 2026

Uh oh!

lokanandaprabhu commented Feb 25, 2026

Uh oh!

mareklibra commented Feb 26, 2026

Uh oh!

Uh oh!

lokanandaprabhu commented Feb 26, 2026

Uh oh!

openshift-cherrypick-robot commented Feb 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

lokanandaprabhu commented Feb 9, 2026 •

edited

Loading

rhdh-gh-app bot commented Feb 9, 2026 •

edited

Loading